Skip to content

cli/command/system: needsServerInfo: add fast-paths#5997

Merged
vvoland merged 1 commit intodocker:masterfrom
thaJeztah:fastpath_needs_serverinfo
Apr 10, 2025
Merged

cli/command/system: needsServerInfo: add fast-paths#5997
vvoland merged 1 commit intodocker:masterfrom
thaJeztah:fastpath_needs_serverinfo

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

We can return early without executing the regular expression or evaluating the template for --format=json or --format='{{json .}}'.

- A picture of a cute animal (not mandatory but encouraged)

We can return early without executing the regular expression or evaluating
the template for `--format=json` or `--format='{{json .}}'`.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Apr 10, 2025
@thaJeztah thaJeztah added this to the 28.1.0 milestone Apr 10, 2025
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 10, 2025

Codecov Report

Attention: Patch coverage is 44.44444% with 5 lines in your changes missing coverage. Please review.

Project coverage is 59.09%. Comparing base (ac375ca) to head (9325743).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5997      +/-   ##
==========================================
- Coverage   59.10%   59.09%   -0.02%     
==========================================
  Files         355      355              
  Lines       29747    29751       +4     
==========================================
- Hits        17583    17582       -1     
- Misses      11189    11193       +4     
- Partials      975      976       +1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vvoland vvoland merged commit bcd9c88 into docker:master Apr 10, 2025
98 checks passed
@thaJeztah thaJeztah deleted the fastpath_needs_serverinfo branch April 10, 2025 10:01
@thaJeztah thaJeztah self-assigned this May 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants